-
Couldn't load subscription status.
- Fork 19
PED10891-Ansible-basics #544
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@dariavladykina please review thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Amrita,
so far these suggestions. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Amrita, please see some linguistic things for the content. Also, I couldn't review the DC-SAP-Ansible article as it built the SLES one for me. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Amrita, please see some linguistic things for the content. Also, I couldn't review the DC-SAP-Ansible article as it built the SLES one for me. Thanks!
|
There are no changes for SAP its the same , equivalent of reviewing SLES :) the confusion may stem from the "wrong file placement" which i commented above |
PR creator: Description
Describe the overall goals of this pull request.
PR creator: Are there any relevant issues/feature requests?
PR reviewer: Checklist for editorial review
Apart from the usual checks, please double-check also the following: